Skip to content

Add the gha-log-uploader lambda - #8591

Open
huydhn wants to merge 9 commits into
gh/huydhn/2/basefrom
gh/huydhn/2/head
Open

Add the gha-log-uploader lambda#8591
huydhn wants to merge 9 commits into
gh/huydhn/2/basefrom
gh/huydhn/2/head

Conversation

@huydhn

@huydhn huydhn commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Stack from ghstack (oldest at bottom):

Impact: none yet -- nothing invokes this function
Risk: low

What

A new lambda that downloads a completed GitHub Actions job log and archives it
to s3://ossci-raw-job-status/log/[<repo>/]<job_id>. It takes a direct invoke
payload ({repo, job_id, conclusion}) and has no API Gateway integration and no
function URL, so lambda:InvokeFunction is the only way in.

The log-download logic is copied from github-status-test: App installation
token minting, the PAT pool fallback, the per-repo cool-off cache, gzip, and the
S3 key scheme. Not copied: the raw event archive, the API Gateway event parsing,
the synthetic backfill action branch, and the classifier ping.

[ghstack-poisoned]
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 20, 2026
@vercel

vercel Bot commented Aug 20, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
torchci Ignored Ignored Aug 21, 2026 9:24pm

Request Review

huydhn added 2 commits August 20, 2026 13:01
[ghstack-poisoned]
[ghstack-poisoned]
huydhn added 3 commits August 20, 2026 15:17
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
@huydhn
huydhn marked this pull request as ready for review August 20, 2026 23:54
[ghstack-poisoned]

@izaitsevfb izaitsevfb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, but consider the nits (non-blocking)

Comment thread aws/lambda/gha-log-uploader/lambda_function.py
Comment thread aws/lambda/gha-log-uploader/lambda_function.py
Comment thread aws/lambda/gha-log-uploader/lambda_function.py
Comment thread aws/lambda/gha-log-uploader/README.md
Comment thread aws/lambda/gha-log-uploader/lambda_function.py
huydhn added 2 commits August 21, 2026 14:14
[ghstack-poisoned]
[ghstack-poisoned]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants